Skip to content

Update documentation for editing post's mm_blocks_actions - #1517

Merged
sbishel merged 2 commits into
masterfrom
MM-69885
Jul 23, 2026
Merged

Update documentation for editing post's mm_blocks_actions#1517
sbishel merged 2 commits into
masterfrom
MM-69885

Conversation

@sbishel

@sbishel sbishel commented Jul 21, 2026

Copy link
Copy Markdown
Member

Summary

  • Added an "Updating and removing actions" section covering:
    • An integration session (bot account, personal access token, or OAuth app) may add, replace, or remove mm_blocks_actions on a post it authored, via the update/patch post endpoints; other sessions can edit the message but not another author's actions.
    • An update that omits mm_blocks_actions preserves the existing actions, so a message-only edit never wipes buttons.
    • Removing a button's mmaction:// link from the message revokes its action — the entry is pruned and later clicks return a not-found error.
  • Replaced the stale plugin-update sentence (which described removal as "restricted for non-integration sessions") with a pointer to the new section.

Related

mattermost/mattermost#37583

Ticket Link

Fixes: https://mattermost.atlassian.net/browse/MM-69885

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown

Newest code from mattermost has been published to preview environment for Git SHA 24eb848

@coderabbitai

coderabbitai Bot commented Jul 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7db18330-7f95-427b-a1c1-48e6dd80532d

📥 Commits

Reviewing files that changed from the base of the PR and between 24eb848 and 3dad8a0.

📒 Files selected for processing (1)
  • site/content/integrate/reference/markdown-actions/_index.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • site/content/integrate/reference/markdown-actions/_index.md
📝 Walkthrough

Walkthrough

The markdown actions reference now documents plugin updates through API.UpdatePost, ownership rules for action changes, behaviour when mm_blocks_actions is omitted, and revocation results.

Changes

Markdown action update rules

Layer / File(s) Summary
Document action update and revocation behaviour
site/content/integrate/reference/markdown-actions/_index.md		 Updates the plugin guidance and adds rules for editing, preserving, removing, and subsequently clicking markdown actions.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: documentation updates for editing post mm_blocks_actions.
Description check ✅ Passed The description accurately describes the documentation changes and is clearly related to the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch MM-69885

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

coderabbitai[bot]
coderabbitai Bot previously requested changes Jul 21, 2026
View reviewed changes

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@site/content/integrate/reference/markdown-actions/_index.md`:
- Around line 193-194: Update the `mm_blocks_actions` omission guidance in the
Markdown actions reference: state that omitted fields are treated as blank and
callers must include the existing actions to preserve them, rather than claiming
omission keeps them. Leave the author/session and endpoint guidance unchanged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ce977891-9631-41d0-a6c9-af3856873635

📥 Commits

Reviewing files that changed from the base of the PR and between c20eda5 and 24eb848.

📒 Files selected for processing (1)
  • site/content/integrate/reference/markdown-actions/_index.md

Comment thread site/content/integrate/reference/markdown-actions/_index.md Outdated
@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown

Newest code from mattermost has been published to preview environment for Git SHA 3dad8a0

@sbishel

sbishel commented Jul 22, 2026

Copy link
Copy Markdown
Member Author

@CodeRabbit review

@coderabbitai

coderabbitai Bot commented Jul 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@sbishel
sbishel requested a review from larkox July 22, 2026 15:50
@sbishel
sbishel dismissed coderabbitai[bot]’s stale review July 23, 2026 14:18

completed changes

@sbishel

sbishel commented Jul 23, 2026

Copy link
Copy Markdown
Member Author

/update-branch

@mattermost-build

mattermost-build commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Error trying to update the PR.
Please do it manually.

@sbishel
sbishel merged commit 9755670 into master Jul 23, 2026
7 checks passed
@sbishel
sbishel deleted the MM-69885 branch July 23, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@larkox larkox larkox approved these changes
@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sbishel @mattermost-build @larkox